Skip to content

Add require_ack_response option#49

Open
okkez wants to merge 4 commits into
fluent:masterfrom
okkez:require-ack-response
Open

Add require_ack_response option#49
okkez wants to merge 4 commits into
fluent:masterfrom
okkez:require-ack-response

Conversation

@okkez

@okkez okkez commented Mar 8, 2016

Copy link
Copy Markdown
Contributor

No description provided.

@okkez

okkez commented Mar 8, 2016

Copy link
Copy Markdown
Contributor Author

This is separated from #47
Merge this after #50, please.

@okkez okkez force-pushed the require-ack-response branch from d9eab0f to 703157a Compare March 16, 2017 05:15
@okkez okkez force-pushed the require-ack-response branch from 703157a to e32e845 Compare May 8, 2017 06:30
@okkez okkez force-pushed the require-ack-response branch from e32e845 to c05c2ee Compare July 26, 2017 06:28
@okkez okkez force-pushed the require-ack-response branch from c05c2ee to a52820a Compare September 27, 2018 05:49
Signed-off-by: Kenji Okimoto <okimoto@clear-code.com>
@okkez okkez force-pushed the require-ack-response branch from a52820a to 3ac3e13 Compare September 27, 2018 05:50
@okkez

okkez commented Sep 27, 2018

Copy link
Copy Markdown
Contributor Author

Rebased.

@duanshiqiang

Copy link
Copy Markdown

ack is really necessary if user want to deliver the log with at-least-once guarantee. Is anyone still care about implementing this??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants